Table: improve tile mode#2102
Merged
fschinkel merged 2 commits intoreleases/26.2from Apr 27, 2026
Merged
Conversation
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
411586d to
c572753
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
from
f59e969 to
4d1fb9b
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
c572753 to
a8d6765
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
from
4d1fb9b to
cd9afd0
Compare
cguglielmo
reviewed
Mar 31, 2026
Member
cguglielmo
left a comment
cguglielmo
left a comment
There was a problem hiding this comment.
There is a typo in the commit message:
ensures that null values are removed from its list of tiles.
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
2 times, most recently
from
a2d21ed to
c27b01c
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
from
cd9afd0 to
a74a3d8
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
c27b01c to
99847cc
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
2 times, most recently
from
5b3cdf8 to
ab26bfb
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
1e9ab18 to
b040482
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
from
e026624 to
5195f7c
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
2 times, most recently
from
05be66b to
0b202d8
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
3 times, most recently
from
bf0981c to
db901d7
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
0b202d8 to
8aadf6d
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
3 times, most recently
from
b09b277 to
3e6ab40
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
8aadf6d to
d4dc5d4
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
from
3e6ab40 to
d2445b1
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
d4dc5d4 to
dbc4991
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
2 times, most recently
from
b42ef9c to
f2c9dde
Compare
fschinkel
force-pushed
the
features/fschinkel/26.2/hybrid-manager-dispose-widgets
branch
from
64172b2 to
7c86766
Compare
Base automatically changed from
features/fschinkel/26.2/hybrid-manager-dispose-widgets
to
releases/26.2
April 27, 2026 08:44
There may not always be a tile for a row. Therefore, implement Table.createTiles more lenient in order to avoid null pointer exceptions. In addition, the TableTileGridMediator ensures that null values are removed from its list of tiles. The list of tiles of the TableTileGridMediator needs to be unique as the tiles will be rendered one after another. If there are duplicates in the list of tiles an 'Already rendered'-error will be thrown if the second occurrence is rendered. Therefore, the TableTileGridMediator removes all duplicates from its list of tiles. 379008
The Scout JS implementation of the sort lookup call for tables in tile mode uses lookup row icons to indicate whether a selected entry sorts ascending or descending while the Scout Classic implementation has added these icons to its text. Align these two implementations and use the style from Scout JS as it looks cleaner. 379008
fschinkel
force-pushed
the
features/fschinkel/26.2/configured-tiles
branch
from
f2c9dde to
097f6e1
Compare
cguglielmo
approved these changes
Apr 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Table: improve tile mode
There may not always be a tile for a row. Therefore, implement Table.createTiles more lenient in order to avoid null pointer exceptions. In addition, the TableTileGridMediator ensures that null values are removed from its list of tiles.
The list of tiles of the TableTileGridMediator needs to be unique as the tiles will be rendered one after another. If there are duplicates in the list of tiles an 'Already rendered'-error will be thrown if the second occurrence is rendered. Therefore, the TableTileGridMediator removes all duplicates from its list of tiles.
379008
Align sort lookup call for table in tile mode
The Scout JS implementation of the sort lookup call for tables in tile mode uses lookup row icons to indicate whether a selected entry sorts ascending or descending while the Scout Classic implementation has added these icons to its text. Align these two implementations and use the style from Scout JS as it looks cleaner.
379008